Skip to content

Modernize python-dev skill to use uv exclusively#134

Merged
calreynolds merged 1 commit intomainfrom
docs/python-dev
Feb 26, 2026
Merged

Modernize python-dev skill to use uv exclusively#134
calreynolds merged 1 commit intomainfrom
docs/python-dev

Conversation

@robkisk
Copy link
Collaborator

@robkisk robkisk commented Feb 18, 2026

Summary

  • Rewrote Environment Management section in the python-dev skill to use uv exclusively — removed all references to pip, python3 -m venv, conda, requirements.txt, and manual venv activation
  • Added dedicated Code Quality subsections for Ruff (linting & formatting) and Pyright (type checking) with runnable command examples
  • Updated Best Practices summary to reflect the modern toolchain (uv, Ruff, Pyright)

Motivation

The skill previously contained conflicting guidance — it mentioned uv alongside venv, pip install, and requirements.txt, leaving ambiguity about which workflow to follow. This aligns the skill with the Astral.sh toolchain (uv + ruff) that the project already uses.

What changed

Section Before After
Dependency Management uv or venv, check .venv, activate uv exclusively, pyproject.toml only
Environment Setup Shell script with venv/pip/uv pip uv sync / uv run / uv add examples
Script Execution "Activate venv, use python3" uv run — no activation needed
Code Style Single Ruff bullet Full Ruff section (check, fix, format)
Type Checking Not mentioned Pyright section
Best Practices #6 "uv or venv, activate before use" uv exclusively, Ruff, Pyright

What was removed

  • pip install -r requirements.txt
  • python3 -m venv .venv / source .venv/bin/activate
  • uv pip install (deprecated subcommand pattern)
  • requirements.txt as a dependency source

Test plan

  • Verify no remaining references to pip, venv (as a tool), or requirements.txt in the skill file
  • Confirm markdown renders correctly (no broken code fences)
  • Validate skill passes .github/scripts/validate_skills.py in CI

@calreynolds calreynolds self-requested a review February 26, 2026 14:50
@calreynolds calreynolds merged commit cf8d6ea into main Feb 26, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants